Conversation
add Test mondodb
add Last version test
| import com.mongodb.casbah.Imports._ | ||
| import com.mongodb.casbah.MongoClient | ||
|
|
||
| class MongoContext(val config: MongoConfig) { |
There was a problem hiding this comment.
С конфигами вы очень намудрили.
Не понял смысла выносить отдельно DefaultConfigKeys и DefaultConfigValues. Обычно это всё задаётся в одном месте, в данном случае application.conf файле. У вас получилось очень размазано.
Сам файл поддерживает возможность задавать дефолтные значения, хардкодить их где-то еще не надо. Также нужно оставить возможность переопределять их переменными среды (environment variables).
Более того, хост и порт из config объекта который сюда передаётся нигде не используются! У меня блин час ушёл пока настраивал докер и не мог понять почему адрес не переопределяется :)
There was a problem hiding this comment.
DefaultConfugKeys используется, чтобы вычитать из application.conf значения. DefaultConfigValues используется, чтобы подставить дефолтное значение, если в application.conf что-то не задано. А так все вычитывается из application.conf, да
There was a problem hiding this comment.
Мой поинт в том, что выносить в переменные значения ключей из application.conf не стоит. Также не стоит хранить дефолтные значения в переменных, синтаксис application.conf позволяет прописать их прямо там. Ну и самое странное, что некоторые из настроек которые вы задали вообще не используются.
add DataSet MongoDB
add test LastVersion
| songsSet.toSeq | ||
| } | ||
|
|
||
| def findWordsDefinitions(version: Option[Int] = None): Seq[Map[Int, String]] = { |
There was a problem hiding this comment.
Мне кажется, что findWordsDefinitions и findSongs можно выразить через одну и ту же базовую функцию.
5 participants
No description provided.